-
Notifications
You must be signed in to change notification settings - Fork 61
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added Twitter Hook #84
base: master
Are you sure you want to change the base?
Conversation
the widget box currently looks too big how about setting a max-height for the widget. I think showing about 2 tweets at a time would be just fine. @Muljayan |
@Muljayan nice work : ) could you display only one tweet and try to keep it as close as possible to this deigns https://github.com/ossn/fixme/blob/master/screens/desktop/Mozilla_Fixme_Screen01_Homepage_A.png ? |
@himankpathak thanks for the suggestion :) |
@Muljayan that's a good idea, what's the limit for the twitter api? |
It says 900-1500 requests per window. Where one window is 15mins . So i think if we don't have 900-1500 people using the app in a span of 15 mins at once we should be fine. |
Is anyone working on this right now? |
As far as I know, no it would be nice to have an implementation which is closer to the designs https://github.com/ossn/fixme/blob/master/screens/desktop/Mozilla_Fixme_Screen01_Homepage_A.png |
closes #14
Included a twitter hook